Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Several Naming Fixes for Netflix and Amazon #13

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

saurabhshri
Copy link

Alternate naming using og:title meta property and slicing of extra non required words.

The property meta tag contains name of the video <meta property="og:title" content="Watch House of Cards Online | Netflix"/>

This fix extracts that and clips the string to obtain title of the video. (Probable fix for : abhishek-vinjamoori#5)
The property meta tag contains name of the video <meta property="og:title" content="Watch House of Cards Online | Netflix"/>

This fix extracts that and clips the string to obtain title of the video. (Probable fix for : abhishek-vinjamoori#5)
In case twitter tag fails, added og:title as backup.
Sliced words -"Watch " and " Amazon Video" for better naming.
Checks and replaces "Watch" "Online | Netflix" only if present.
Replaces "Watch" and "- Amazon Video" only if present.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant